Skip to content

chore: prepare 0.13.2 release#44

Closed
isasmendiagus wants to merge 1 commit into
mainfrom
chore/prepare-release-0.13.2
Closed

chore: prepare 0.13.2 release#44
isasmendiagus wants to merge 1 commit into
mainfrom
chore/prepare-release-0.13.2

Conversation

@isasmendiagus

@isasmendiagus isasmendiagus commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What

Release preparation for 0.13.2.

Why

PR #43 (disable OkHttp default read timeout) was merged to main without a version bump or CHANGELOG entry. Because the current pom version (0.13.1) already has a matching tag (v0.13.1), tools/get_next_version.sh aborts and the release pipeline cannot produce a new tag. This bumps to 0.13.2 so the release can proceed.

Release steps (after merge)

  1. Merge this PR to main.
  2. Run the Repo Version Tagging workflow (workflow_dispatch) with run_for_real = true → creates and pushes tag v0.13.2.
  3. The tag push triggers Publish → build/test → deploy to Maven Central + native binaries + draft GitHub Release.
  4. Review and publish the draft release.

Verification

The underlying fix was verified with a MockWebServer test:

  • Old client (callTimeout only) dies at ~10s on a slow read (the bug).
  • PR code (readTimeout=0, callTimeout=60s) completes a 15s slow read successfully.
  • callTimeout still bounds the request: with callTimeout=5s the call aborts at ~5s even with read timeout disabled (roof timeout preserved).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes

    • Improved reliability for long-running scans and large payloads by preventing premature timeouts during requests.
    • Requests now complete based on the overall call timeout, reducing unexpected failures on slower operations.
  • Chores

    • Updated the release version to 0.13.2.

Bump version to 0.13.2 and document the OkHttp read timeout fix
(disabled default read timeout; requests remain bounded by callTimeout).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request bumps the project version from 0.13.1 to 0.13.2 in pom.xml and adds a corresponding CHANGELOG.md entry describing a fix that disables OkHttp's default read timeout to prevent premature failures on large payloads during long-running scans.

Changes

Release version bump

Layer / File(s) Summary
Version and changelog update
pom.xml, CHANGELOG.md
Project version updated to 0.13.2 and changelog documents disabling OkHttp's default read timeout so long-running scans are bounded only by the overall call timeout.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Related Issues: None found in provided context.

Related PRs: None found in provided context.

Suggested labels: release, documentation

Suggested reviewers: None specified.

🐰

A version hops up, one small step,
The changelog notes what timeouts kept,
No more waits cut short so soon,
Long scans now finish 'neath their own moon.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: preparing the 0.13.2 release with a version bump and changelog update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/prepare-release-0.13.2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@CHANGELOG.md`:
- Around line 10-13: The changelog entry for 0.13.2 is missing its version
comparison link reference, so add the corresponding [0.13.2] link in the
reference section at the bottom of CHANGELOG.md. Keep the existing entry text
unchanged, and mirror the format used by other version labels in the changelog
so the new reference resolves consistently.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 92c53aa3-602c-461b-8dc3-9d73cfc4c9f6

📥 Commits

Reviewing files that changed from the base of the PR and between c37e2dd and 5a15315.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • pom.xml

Comment thread CHANGELOG.md
Comment on lines +10 to +13
## [0.13.2] - 2026-07-01
### Fixed
- Disabled OkHttp's default read timeout so long-running scans are bounded only by the overall call timeout, preventing premature failures on large payloads.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Add missing version comparison link for [0.13.2].

The changelog entry content is accurate and matches the implementation in ScanApi.java (disabling readTimeout while keeping callTimeout). However, the [0.13.2] reference link is missing at the bottom of the file. Add it to maintain consistency with existing versions and ensure the reference resolves.

 [0.13.1]: https://github.com/scanoss/scanoss.java/compare/v0.13.0...v0.13.1
+[0.13.2]: https://github.com/scanoss/scanoss.java/compare/v0.13.1...v0.13.2
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
## [0.13.2] - 2026-07-01
### Fixed
- Disabled OkHttp's default read timeout so long-running scans are bounded only by the overall call timeout, preventing premature failures on large payloads.
[0.13.1]: https://github.com/scanoss/scanoss.java/compare/v0.13.0...v0.13.1
[0.13.2]: https://github.com/scanoss/scanoss.java/compare/v0.13.1...v0.13.2
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@CHANGELOG.md` around lines 10 - 13, The changelog entry for 0.13.2 is missing
its version comparison link reference, so add the corresponding [0.13.2] link in
the reference section at the bottom of CHANGELOG.md. Keep the existing entry
text unchanged, and mirror the format used by other version labels in the
changelog so the new reference resolves consistently.

@isasmendiagus

Copy link
Copy Markdown
Contributor Author

Superseded by the 0.13.1 release-prep PR. 0.13.1 was never actually tagged/released (latest tag is v0.13.0), so we release the read-timeout fix as part of 0.13.1 instead of bumping to 0.13.2 and leaving a version gap.

@isasmendiagus isasmendiagus deleted the chore/prepare-release-0.13.2 branch July 1, 2026 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant